Skip to content

GH-95150: Use position and exception tables for code hashing and equality#95509

Merged
brandtbucher merged 5 commits into
python:mainfrom
brandtbucher:code-equality
Aug 1, 2022
Merged

GH-95150: Use position and exception tables for code hashing and equality#95509
brandtbucher merged 5 commits into
python:mainfrom
brandtbucher:code-equality

Conversation

@brandtbucher

@brandtbucher brandtbucher commented Jul 31, 2022

Copy link
Copy Markdown
Member

This requires relaxing a few tests, which are probably a bit too restrictive.

@brandtbucher brandtbucher added type-bug An unexpected behavior, bug, or error interpreter-core (Objects, Python, Grammar, and Parser dirs) needs backport to 3.11 only security fixes labels Jul 31, 2022
@brandtbucher brandtbucher requested a review from pablogsal July 31, 2022 20:51
@brandtbucher brandtbucher self-assigned this Jul 31, 2022

@markshannon markshannon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks for the fix.

@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @brandtbucher for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

@bedevere-bot

Copy link
Copy Markdown

GH-95541 is a backport of this pull request to the 3.11 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 1, 2022
…d equality (pythonGH-95509)

(cherry picked from commit c7e5bba)

Co-authored-by: Brandt Bucher <brandtbucher@gmail.com>
miss-islington added a commit that referenced this pull request Aug 1, 2022
…lity (GH-95509)

(cherry picked from commit c7e5bba)

Co-authored-by: Brandt Bucher <brandtbucher@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

interpreter-core (Objects, Python, Grammar, and Parser dirs) type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants